ci: trigger builds on ros2_medkit_updated dispatch#36
Conversation
There was a problem hiding this comment.
Pull request overview
Updates CI to allow ros2_medkit to trigger selfpatch_demos docker image build verification via repository_dispatch, and adds a small traceability summary when that happens. Also adjusts demo runner scripts to consolidate docker-compose invocation and adds cleanup behavior on build failure, plus aligns the MoveIt demo Dockerfile with the other demos’ ROS2MedkitCompat.cmake handling.
Changes:
- Add
repository_dispatchtrigger (ros2_medkit_updated) to CI and show source metadata in the job summary when dispatch-triggered. - Refactor demo
run-demo.shscripts to select aCOMPOSE_CMDand add a “build failed” cleanup path. - Update MoveIt Pick-and-Place Dockerfile to install
ROS2MedkitCompat.cmakeinto${COLCON_WS}/cmake.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| demos/turtlebot3_integration/run-demo.sh | Consolidates compose command selection; adds cleanup on build failure. |
| demos/sensor_diagnostics/run-demo.sh | Same compose refactor; adds cleanup on build failure (with a ShellCheck concern). |
| demos/moveit_pick_place/run-demo.sh | Same compose refactor; adds cleanup on build failure. |
| demos/moveit_pick_place/Dockerfile | Moves ROS2MedkitCompat.cmake into workspace cmake dir like other demos. |
| .github/workflows/ci.yml | Adds repository_dispatch trigger and dispatch-only “source” step summary output. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bburda
force-pushed
the
feat/repository-dispatch-trigger
branch
from
c170348 to
20998f2
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…, add workflow permissions
Description
Adds a
repository_dispatchtrigger (ros2_medkit_updatedevent type) soros2_medkitCI can notifyselfpatch_demosto run a docker-build verification whenever ros2_medkit merges to main. Without this, breakages in ros2_medkit that affect the demo images go undetected until someone manually runs selfpatch_demos CI.Also adds a
Show triggering sourcestep as the first step in thedocker-buildjob. When triggered byrepository_dispatch, it writes a step summary with the source commit SHA and a link to the ros2_medkit run - making it easy to trace a failed demo build back to the ros2_medkit change that caused it.Related Issue
closes selfpatch/ros2_medkit#224
Checklist